Skip to content

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969

Merged
slipher merged 1 commit intoDaemonEngine:masterfrom
slipher:maxlights
May 9, 2026
Merged

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969
slipher merged 1 commit intoDaemonEngine:masterfrom
slipher:maxlights

Conversation

@slipher
Copy link
Copy Markdown
Member

@slipher slipher commented May 8, 2026

No description provided.

@illwieckz
Copy link
Copy Markdown
Member

illwieckz commented May 8, 2026

Side note: we haven't fixed it yet, but when you open a PR with a branch from a GitHub fork, it doesn't run Azure CI anymore.

@slipher
Copy link
Copy Markdown
Member Author

slipher commented May 8, 2026

OK I fixed it. Now I just need to delete the duplicate version of the pipeline :)

@illwieckz
Copy link
Copy Markdown
Member

Nice!

@slipher
Copy link
Copy Markdown
Member Author

slipher commented May 8, 2026

It was this ("Require a team member's comment"). It's an organization-scope setting so it ought to be fixed too for Crunch and any other repos we have.

image

@illwieckz
Copy link
Copy Markdown
Member

I had also tested that… but it didn't work. Weird. Anyway, thanks for having fixed it!

@illwieckz
Copy link
Copy Markdown
Member

illwieckz commented May 8, 2026

This doesn't fix the realtime lights on ZX C-1190.

Copy link
Copy Markdown
Member

@illwieckz illwieckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anyway, more fixes are welcome, LGTM.

@slipher slipher merged commit 0cddd7e into DaemonEngine:master May 9, 2026
14 checks passed
@slipher slipher deleted the maxlights branch May 9, 2026 19:48
@illwieckz illwieckz mentioned this pull request May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants